Can someone help me understand why this implementation strategy is needed/better than using standard Eloquent Accessor caching?

Can someone help me understand why this implementation strategy is needed/better than using standard Eloquent Accessor caching?

Mainly for your work on Hashtags, but more generally for any logic that requires reading actual new data from a dirty state in the same request. In this instance, the QuestionObserver update method is passed the model with cached values, rather than the updated value (Attribute accessors don't cache bust) & dirty state triggers the Hashtag parser is parsing the old value, so we needed a way to bust the cache within the same request when the value is dirty & the property is called.

Obviously I wanted to understand what was going on, so I looked and found that the framework is just not designed to facilitate anything outside of returning a single Attribute::make() call within the accessor method, so conditionals on dirty or something like that is out of the question. HasAtrributes trait uses reflection to determine if it should be cached, so it's not designed to handle it & actually triggers infinite recursion on a conditional and kills the application.

There might be an opportunity for a framework PR looking into the option to cache bust if the value is dirty, but for now this is our best solution.

Interestingly too, the defining accessors using the Attribute class adds a bit of overhead, which on benchmarking doesn't give us any performance gain over the current repeated parsing, obviously that would change as we add additional parser, but if we use the old syntax the benchmarks show an improvement.

Can someone help me understand why this implementation strategy is needed/better than using standard Eloquent Accessor caching?

Mainly for your work on Hashtags, but more generally for any logic that requires reading actual new data from a dirty state in the same request. In this instance, the QuestionObserver update method is passed the model with cached values, rather than the updated value (Attribute accessors don't cache bust) & dirty state triggers the Hashtag parser is parsing the old value, so we needed a way to bust the cache within the same request when the value is dirty & the property is called.

Obviously I wanted to understand what was going on, so I looked and found that the framework is just not designed to facilitate anything outside of returning a single Attribute::make() call within the accessor method, so conditionals on dirty or something like that is out of the question. HasAtrributes trait uses reflection to determine if it should be cached, so it's not designed to handle it & actually triggers infinite recursion on a conditional and kills the application.

There might be an opportunity for a framework PR looking into the option to cache bust if the value is dirty, but for now this is our best solution.

Interestingly too, the defining accessors using the Attribute class adds a bit of overhead, which on benchmarking doesn't give us any performance gain over the current repeated parsing, obviously that would change as we add additional parser, but if we use the old syntax the benchmarks show an improvement.

Thanks for the answer Cam. Very interesting. Does sound like a potential PR for the framework. I might work on that if I have time.

Out of curiosity, I tried a hack, and it appears to work lol. Obviously we still have the performance concern as you stated but a funny hack nonetheless.

    protected function answer(): Attribute
    {
        return Attribute::make(
            get: fn (?string $rawAnswer): ?string => $rawAnswer ? (new ParsableContent())->parse($rawAnswer) : null,
            set: function (?string $newAnswer): ?string {
                unset($this->attributeCastCache['answer']); // 👀

                return $newAnswer;
            },
        )->shouldCache();
    }

    $question = \App\Models\Question::factory()->create([
        'answer' => 'Original answer @foo',
    ]);

    $originalParsedAnswer = $question->answer; // parsed with @foo link

    $question->answer = 'This is some new content #hashtag';

    $newParsedAnswer = $question->answer; // parsed with #hashtag link